-
Notifications
You must be signed in to change notification settings - Fork 70
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
VACMS-19772: update composer dependencies first round. #19955
Conversation
Checking composer.lock changes... |
Checking composer.lock changes... |
# Conflicts: # composer.lock
Checking composer.lock changes... |
Checking composer.lock changes... |
Checking composer.lock changes... |
Checking composer.lock changes... |
Keeping track of all modules updated or not in this sheet: https://docs.google.com/spreadsheets/d/1exMktEjiY9CaKtzvh0Qi-F8ffuDz-jJ5LXMuHVx4cu0/edit?usp=sharing |
…om/department-of-veterans-affairs/va.gov-cms into 19772-update-composer-dependencies
Checking composer.lock changes... |
Checking composer.lock changes... |
Cypress Accessibility Violations
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!!
Description
This PR encapsulates the work for updating modules and exporting configurations related. A full list of the updates are here: https://docs.google.com/spreadsheets/d/1exMktEjiY9CaKtzvh0Qi-F8ffuDz-jJ5LXMuHVx4cu0/edit?usp=sharing
Relates to #19772
Testing done
Local logs and spot checking.
QA steps
What needs to be checked to prove this works?
Look at list of composer modules updated to see if it matches expectations.
What needs to be checked to prove it didn't break any related things?
Site shouldn't change any functionality or look.
What variations of circumstances (users, actions, values) need to be checked?
Definition of Done
Select Team for PR review
CMS Team
Public websites
Facilities
User support
Accelerated Publishing